Skip to content

provider/azurerm: Adding azurerm_virtual_machine resource#5514

Merged
jen20 merged 3 commits into
masterfrom
f-arm-virtual-machine
Mar 17, 2016
Merged

provider/azurerm: Adding azurerm_virtual_machine resource#5514
jen20 merged 3 commits into
masterfrom
f-arm-virtual-machine

Conversation

@stack72

@stack72 stack72 commented Mar 8, 2016

Copy link
Copy Markdown
Contributor

Adds the azurerm_virtual_machine resource. We currently have tests for basic linux and windows machines. More tests will be added later. This is the initial work that will allow us to create machines :)

@avdhoot

avdhoot commented Mar 17, 2016

Copy link
Copy Markdown

Any progress? any ETA. Please guys. :(

@stack72 stack72 force-pushed the f-arm-virtual-machine branch from 935e1bb to 4f8bbdd Compare March 17, 2016 13:09
@stack72 stack72 changed the title [WIP] provider/azurerm: Adding azurerm_virtual_machine resource provider/azurerm: Adding azurerm_virtual_machine resource Mar 17, 2016
@jen20 jen20 force-pushed the f-arm-virtual-machine branch from 4f8bbdd to 808ce45 Compare March 17, 2016 15:09
@jen20

jen20 commented Mar 17, 2016

Copy link
Copy Markdown
Contributor

Acceptance test run:

make testacc TEST=./builtin/providers/azurerm TESTARGS="-run TestAccAzureRMVirtualMachine"
==> Checking that code complies with gofmt requirements...
/Users/James/Code/go/bin/stringer
go generate $(go list ./... | grep -v /vendor/)
TF_ACC=1 go test ./builtin/providers/azurerm -v -run TestAccAzureRMVirtualMachine -timeout 120m
=== RUN   TestAccAzureRMVirtualMachine_basicLinuxMachine
--- PASS: TestAccAzureRMVirtualMachine_basicLinuxMachine (592.77s)
=== RUN   TestAccAzureRMVirtualMachine_basicWindowsMachine
--- PASS: TestAccAzureRMVirtualMachine_basicWindowsMachine (851.18s)
PASS
ok      github.com/hashicorp/terraform/builtin/providers/azurerm    1443.968s

@jen20

jen20 commented Mar 17, 2016

Copy link
Copy Markdown
Contributor

This LGTM @stack72! I've rebased it and merged to master and will push shortly.

@jen20 jen20 merged commit 808ce45 into master Mar 17, 2016
@jen20

jen20 commented Mar 17, 2016

Copy link
Copy Markdown
Contributor

Landed in and: 808ce45, 4db3242, cb10904.

Thanks @stack72 - this is great work on this API!

@jen20 jen20 deleted the f-arm-virtual-machine branch March 17, 2016 15:41
@lmeyemezu

Copy link
Copy Markdown

Hi,
is Azure RM virtual machine yet available?
Regards

@stack72

stack72 commented Mar 18, 2016

Copy link
Copy Markdown
Contributor Author

Hi @lmeyemezu,

This has already been merged and will be made available in Terraform 0.6.14

Paul

@lmeyemezu

Copy link
Copy Markdown

Hi @stack72 ,
Thanks.

@ghost

ghost commented Apr 27, 2020

Copy link
Copy Markdown

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants